Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support optional Mixpanel init config options #244

Merged

Conversation

vladucu
Copy link
Contributor

@vladucu vladucu commented Aug 4, 2020

Adds support for other optional config options for Mixpanel's init call.
See https://developer.mixpanel.com/docs/javascript-full-api-reference#mixpanelinit for more

@jherdman
Copy link
Contributor

Are you amenable to tackling the feature requested in ##229 too?

@vladucu
Copy link
Contributor Author

vladucu commented Aug 24, 2020

Are you amenable to tackling the feature requested in ##229 too?

This change should allow customizing api_host too, while keeping the existing default. Let me know if this is not what you were referring to

@JamesS-M
Copy link

JamesS-M commented Jan 5, 2021

What's the status of this?

@jherdman
Copy link
Contributor

jherdman commented Jan 5, 2021

@vladucu this could use a rebase if you have the time. Once you do so I'm happy to merge.

@vladucu
Copy link
Contributor Author

vladucu commented Jan 6, 2021

@jherdman rebased. Thanks!

@jherdman jherdman merged commit d653707 into adopted-ember-addons:master Jan 6, 2021
@jherdman
Copy link
Contributor

jherdman commented Jan 6, 2021

I believe this resolves #229 too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants